Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calibrate qualified peer did tests to acapy PR 2862 #795

Merged

Conversation

nodlesh
Copy link
Contributor

@nodlesh nodlesh commented Apr 15, 2024

This PR accounts for the ACA-Py qualified peer did updates made in PR 2862, openwallet-foundation/acapy#2862

The biggest change here is that we no longer need to restart the agents with different did peer methods as a command line option. Peer did method is now on the API calls themselves. I'm leaving in the code that restarts agents just in case other frameworks require a restart to work with a new peer did method.

Note that test T003-RFC0793 will fail until ACA-Py PR 2891 is merged. openwallet-foundation/acapy#2891

@nodlesh nodlesh requested review from swcurran and dbluhm April 15, 2024 16:13
Copy link
Contributor

@swcurran swcurran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not all tests in ACA-Py runsets, but more than on main.

Approving, but may need more work?

@swcurran swcurran merged commit 6382d49 into openwallet-foundation:main Apr 17, 2024
1 check passed
@nodlesh
Copy link
Contributor Author

nodlesh commented Apr 17, 2024

Besides the last test in this suite waiting for 2891, all the OOB tests are failing and waiting on ACA-Py PR 2897. openwallet-foundation/acapy#2897
If there are more failures than that, then let me know. I am seeing some new didcmmm transport tests failing in last nights run, so I'll investigate that this morning.

@swcurran
Copy link
Contributor

Thanks @nodlesh -- oob pr merged. Can you manually trigger another run of AATH? I'm not sure how to do that in the Actions interface?

@nodlesh
Copy link
Contributor Author

nodlesh commented Apr 17, 2024

Done. Running the acapy-acapy runset.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants